Conversation
supabase/.branches/_current_branch
Outdated
There was a problem hiding this comment.
remove these temp files pls!
supabase/.temp/cli-latest
supabase/.branches/_current_branch
| StripePaymentMethodID string `json:"stripe_payment_method_id" db:"stripe_payment_method_id"` | ||
| CardBrand *string `json:"card_brand,omitempty" db:"card_brand"` | ||
| CardLast4 *string `json:"card_last4,omitempty" db:"card_last4"` | ||
| CardExpMonth *int `json:"card_exp_month,omitempty" db:"card_exp_month"` |
There was a problem hiding this comment.
do we need to store any of this card info or can it be pulled from stripe?
There was a problem hiding this comment.
This is nice to have to show on the frontend and reduce calls to the API
There was a problem hiding this comment.
This feels like it would not be very secure. I think for something like credit card info it would be better to just do the api calls bc our database will not be secure compliant
| return c.Status(fiber.StatusOK).SendString("Welcome to SkillSpark!") | ||
| }) | ||
|
|
||
| stripeAPIKey := os.Getenv("STRIPE_SECRET_TEST_KEY") |
There was a problem hiding this comment.
I think we would not want to name this test_key bc then we'll need to update the code when we switch out of test mode. better to keep it a generic name imo
|
|
||
| func (sc *StripeClient) CreatePaymentIntent(ctx context.Context, input *models.CreatePaymentIntentInput) (*models.CreatePaymentIntentOutput, error) { | ||
|
|
||
| const applicationFeePercentage = 10 // CHANGE THIS TO BE THE APPLICATION FEE PERCENTAGE |
There was a problem hiding this comment.
can be handled later - but i feel like this percent might differ by organization
| guardian_id UUID NOT NULL REFERENCES guardian(id) ON DELETE CASCADE, | ||
| stripe_payment_method_id VARCHAR(255) NOT NULL UNIQUE, | ||
|
|
||
| card_brand VARCHAR(50), |
There was a problem hiding this comment.
Yeah i think we're def not going to want to store any of the card information here
|
|
||
|
|
||
| ALTER TABLE event_occurrence | ||
| ADD COLUMN IF NOT EXISTS price INTEGER NOT NULL DEFAULT 0; -- Price in bhat |
There was a problem hiding this comment.
can we add currency here as well
2 participants
This is what I got so far - I just need to finish testing, implement payment intents in registrations and create webhooks.
Description
[Link to Ticket](insert the link to your ticket inside the parenthesis here)
Please include a summary of the changes and the related issue. Please also
include relevant motivation, context, and images!
How Has This Been Tested?
Please describe the tests that you manually ran to verify your changes (beyond any unit/integration tests written and ran).
Screenshots
Please provide screenshots of manual testing (Scalar, frontend pages, etc.)
Checklist
General
Backend Changes (if applicable)
Frontend Changes (if applicable)
Mobile Screenshots (if applicable)
iOS
[Add iOS screenshots here]
Android
[Add Android screenshots here]